wayland: Wait for output and seat information

When we open the connection, we get the wl_output object,
but we return before all the information such as monitor
geometry has arrived, which causes us to misinform early
users of this information. Do a roundtrip here that causes
us to wait until the information is complete. Do the same
for seats, just in case.
https://bugzilla.gnome.org/show_bug.cgi?id=747471

wayland: Add more debug output

Make gtk-update-icon-cache not fall over leftover temp files

This is a followup to 0fd185fa6de2a89c11b4a28. There is no
good reason to only try again if --force is passed. Do it
always.

See
https://bugzilla.redhat.com/show_bug.cgi?id=1194957

tree view: Avoid a crash with rubberbanding

When a treeview is destroyed while rubberbanding is going
on, we crash because the rb tree is nuked before we want
to access it to stop the rubberbanding. To avoid this crash
end the rubberbanding early in destroy().

See
https://bugzilla.redhat.com/show_bug.cgi?id=1173904

GtkApplication: avoid a crash

It can apparently happen that we get focus in events
on windows after gtk_application_shutdown() has been
called. Avoid an unnecessary crash in this case.

See
https://bugzilla.redhat.com/show_bug.cgi?id=1176339

Revert "image: Optimize non-resize changes"

This reverts commit bae97a4c6bcb1c82d4a5f296c3d428173e31c6e4.

We're still seeing regressions from there, see
http://bugzilla.redhat.com/show_bug.cgi?id=1208183

testgtk: Respect display cursor size limitations

Instead of hardcoding 64, ask the display about the
maximal supported cursor size.

mir: Fix build failure due to changed API

mir_surface_set_event_handler() now takes three arguments.

Fix indicator proximity checks

The coordinate translations here were not working properly
for window widgets inside the scrolled window, as can be
seen e.g. for the horizontal scrollbar of the 'Tree View'
example in gtk3-demo.

https://bugzilla.gnome.org/show_bug.cgi?id=747406

Make indicators pop out when needed

When moving over a non-expanded indicator from the outside, we were
not expanding it, due to on_scrollbar being true. This can be seen
e.g. when moving from the content pane over to the sidebar indicator
in gtk3-demo. We must still ensure that the indicator is expanded
when receiving motion events over the indicator.

https://bugzilla.gnome.org/show_bug.cgi?id=747407

Updated breton translation

docs: Note the side effects of GtkLabel setters

The gtk_label_set_text() and gtk_label_set_markup() functions have
various side effects that ought to be documented, especially for
non-C developers using properties directly.

https://bugzilla.gnome.org/show_bug.cgi?id=747121

file chooser: Ensure a folder is loaded when browsing

Just showing an empty list when the user hits Ctrl-l in
recent files is not very useful. Load the home directory
instead.

file chooser: Open location entry more easily

Show the location entry also when alphanumerics are typed
into the list.

testfilechooser: Improve filter setup

It is not ideal when the default filter filters away
all files, so change things to select the 'all files'
filter by default.

Formatting fix

Formatting fix

testgtk: Set initial cursor

Arrange for the entry and the cursor area to both start out
with the arrow cursor.

testgtk: Redo the cursor example

Use cursor names instead of font cursors, so we can also show
cursors that are not represented in the X cursor font and thus
don't have a value in the GdkCursorType enumeration.

wayland: Don't mix free and g_strdup

While it works most of the time, this is technically
not correct.

testgtk: Update cursor theme on size change

We were connecting to the wrong signal, so we were not picking
up all changes to the spin button value.

GtkListBox: Make model binding introspectable

Introspection can't deal with bare pointers, so annotate
the callback as taking objects. This is the outcome of
discussion in
https://bugzilla.gnome.org/show_bug.cgi?id=746893

gtktextbuffer: small cleanup in clipboard handling

Make code shorter and also more efficient since we move the
selection check out of the loop

https://bugzilla.gnome.org/show_bug.cgi?id=747096

x11/gl: Fix use after free

https://bugzilla.gnome.org/show_bug.cgi?id=747305

Signed-off-by: Olivier Fourdan <fourdan@xfce.org>

Adwaita: stronger entry focus indication

See https://bugzilla.gnome.org/show_bug.cgi?id=746202 for details.

Updated Slovak translation

scrolledwindow: Ignore 0/0 scroll events when possibly cancelling animation

These should be used eventually to start kinetic scrolling, so should definitely
be ignored on cancellation.

https://bugzilla.gnome.org/show_bug.cgi?id=747133

Updated Russian translation

Updated Russian translation

Udated Czech translation

Updated Slovenian translation

scrolledwindow: Remove needless "dragging" field from Indicator struct

The "over" state already stays set while scrollbar dragging happens, there's
no need to double track that.

https://bugzilla.gnome.org/show_bug.cgi?id=746961

scrolledwindow: Check the event widget on captured motion events

This path is only intended to be triggered on events directed towards the
child of the scrolledwindow, so make it explicitly so. This avoids scrollbar
"over" state flashing when dragging finishes within the slider.

https://bugzilla.gnome.org/show_bug.cgi?id=746961

wayland: Set a more correct xdg_surface application id

The "app_id" of a xdg_surface should be the ID that can potentially be
used to get the DBUS name or the .desktop file.

For GtkApplication programs this is often the ID passed when creating the
GtkApplication object, so when available lets use that.

As fallbacks, first try g_get_prgname as it often corresponds to the
basename part of the .dektop file for non-GtkApplication programs.
Otherwise use gdk_get_program_class, even though that string usually
doesn't conform to the expectations of xdg_surface.set_application_id.

https://bugzilla.gnome.org/show_bug.cgi?id=746435

Updated Russian translation

Revert "Adwaita: desaturate selected color for backdrop"

This reverts commit 6f478db2facd8744714b7ab374d2687d777653a3.

As per discussion on bug 746202, the real solution to the case
in bug 704352 is what the reporter suggests.

Updated Ukrainian translation

gtkdnd: Restore widget opacity after fading out

Otherwise the widget is left fully transparent, and will start as such if
the widget is reused.

gtkdnd: Set the icon helper from the drag site onto the context

If the icon happened to come from the drag site icon helper (ie. set through
gtk_drag_source_set_icon*), it would be referenced on the GtkDragInfo,
but not hooked into its context. This results on non visible drag windows,
until set_icon_helper() happened to be called on some path.

roundedbox: Fix a copy/paste error

This caused shadows to not be drawn in certain situations when overlayed
via a popover animation.

https://bugzilla.gnome.org/show_bug.cgi?id=746862

Updated Czech translation

radio button: Fix directional navigation

GtkRadioButton had a deficient copy of the focus sort code
in GtkContainer, causing focus to jump over the next button
in the list. Just use _gtk_container_focus_sort() here,
which fixes the bug _and_ saves 80 lines of code.

https://bugzilla.gnome.org/show_bug.cgi?id=746817

Fix separator in testsplitheaders

gtkprogressbar: fix size allocation

As of 74405cc, progress bars use a new design with values drawn on top
(or to the left) of the through instead of inside of it. This change
brought a number of regressions: the min-horizontal-bar-height and
min-vertical-bar-width style properties are not respected anymore. For
vertical progress bars, the value was drawn too close to the bar and not
centered vertically.

Fix this by respecting the style properties and drawing the value label
at the correct position.

Also, the xspacing and yspacing properties didn't server any apparent
purpose. Change their semantics to mean "the spacing between the label
and the bar". Hence, they only need to be added to the size request when
showing the label. Since we are changing semantics anyway, reduce their
default values from 7 to 2, to avoid and excessive gap.

https://bugzilla.gnome.org/show_bug.cgi?id=746688

make libcanberra support optional

Make it possible to control libcanberra support so we don't risk ending up with
a cyclic dependency when using packages: gtk+ -> libcanberra-gtk3 -> gtk+

https://bugzilla.gnome.org/show_bug.cgi?id=746904

Updated Slovenian translation

Upload Catalan translation

GtkListBox: fix model binding refcount issue

As it is, GtkListBox model binding will work nicely as long as your
create_widget_func returns a floating reference on the newly-created
widget.

If you try to return a full reference (as any higher-level language
would do) then you will leak that reference.

Fix that up by converting any floating references into full references
and then unconditionally releasing the full reference after adding to
the box.

https://bugzilla.gnome.org/show_bug.cgi?id=746893

scrolledwindow: Extend trackpoint heuristic to ALPS trackpoints

These have a different characteristic device name, look for it too
in order to enable trackpoint features.

docs: Add initialization example for GtkGLArea

Show how to safely check for errors when initializing the OpenGL
pipeline.

x11/gl: Do not use GLX_EXT_texture_from_pixmap if not available

If the GLX_EXT_texture_from_pixmap extension is not available when we
did the extensions check, then there's no point in using the backend
specific code paths that rely on it.

W32: Only override window miximized position for CSDed windows

This fixes an issue with non-CSDed windows being improperly positioned
when maximized, which was introduced by the previous commit.

https://bugzilla.gnome.org/show_bug.cgi?id=746821

W32: Correctly set maximized window position

Use screen workarea to *also* set the position of a maximized window,
not just its size. Without this the window position defaults to 0:0
(the topleft corner), which is wrong when taskbar is position along the
top or left edge of the screen.

https://bugzilla.gnome.org/show_bug.cgi?id=746821

bbox: correct documentation typo

Fix an AtkStateType emitted for GtkBooleanCellAccessible

An apparent copy-and-paste error is resulting in ATs being notified of
changes in ATK_STATE_CHECKED when the cell's "sensitive" state changes.

https://bugzilla.gnome.org/show_bug.cgi?id=746784

textview: Fix "Since" versions after backport

Fix for 28063ee2e42e7ce47b7bd5326f2d53875a377d57 which got cherry-picked
to 3.16

demo/glarea: Check errors on GtkGLArea

We should not call OpenGL API if GtkGLArea is in an error state.

https://bugzilla.gnome.org/show_bug.cgi?id=746746

glarea: Check that the context is not NULL

The public API should perform more checks when it comes to an NULL
context.

https://bugzilla.gnome.org/show_bug.cgi?id=746746

x11/gl: Do not try to call MakeCurrent with a NULL context

This should not happen, but better safe than sorry.

https://bugzilla.gnome.org/show_bug.cgi?id=746746

x11/gl: Trap GL context creation

Avoid an X11 error in case the context creation fails, so we can avoid a
crash and pick it up in the upper layers, and display an error.

https://bugzilla.gnome.org/show_bug.cgi?id=746746

filechooser: Use same priority everywhere

I managed to stall recent files today while trying to save a GTK
testcase in glade that contained enough spinning spinners that the CPU
was saturated just redrawing things.

I had to navigate the filesystem!

GtkRange: Check "inverted" property when drawing

The direction in which the slider moves can be inverted by setting the
inverted property. But the draw method does not check this, instead it
checks if the direction of the widget is set to be right to left.

Call the should_invert function in order to determine if the direction
of the range should be inverted. It too checks the widget's direction,
but also checks the "inverted" property, and allows the range to be
drawn inverted even if it is vertically oriented.

https://bugzilla.gnome.org/show_bug.cgi?id=746712

a11y: Only emit name-changed once

We were emitting it twice, once with deleting the old text, once with
inserting the new text.

https://bugzilla.gnome.org/show_bug.cgi?id=746706

a11y: Free the cell text after emitting text-changed:delete.

This is needed because AT-SPI2's ATK bridge exposes the deleted string
to ATs.

https://bugzilla.gnome.org/show_bug.cgi?id=746706

a11y: Refactor code

Put the equality check in front. This allows better detection of when an
insert or delete needs to be emitted.

Also, only emit text-changed:delete if the deleted text is not the empty
string. Only emit text-changed:insert if the inserted text is not the
empty string.

https://bugzilla.gnome.org/show_bug.cgi?id=746706

a11y: Refactor textcellaccessible code

Compute the new text and its length in advance. This way those
computations will not confuse us when they happen in the middle of the
actual action.

https://bugzilla.gnome.org/show_bug.cgi?id=746706

GtkRange: Fix drawing of fill-level

Compute the proportion of the range that should be filled to match the
fill level, and use it to compute the starting point and length of the
area between the slider and the fill level.

https://bugzilla.gnome.org/show_bug.cgi?id=734741

inspector: fix warning about not using the right format for an int64

cssnode: Clear widget path more aggressively

When recomputing CSS, we need a correct widget path in the fallback mode
where we're still using widget paths.
So we need to invalidate it everytime it actually changes, and not just
when emitting the style-updated signal.

Fixes css-match-regions reftest.

Updated Korean translation

x11/gl: Removing some more legacy GL calls

They are not required any more.

http://bugzilla.gnome.org/show_bug.cgi?id=746668

x11/gl: Do not call glEnable/glDisable with GL_TEXTURE_2D

These are fixed pipeline functions, and Mesa will warn (if debugging
messages are enabled). At best, they are poinless calls.

Updated Basque language

Updated Basque language

Updated Basque language

Fix a minor typo in translatable string

Updated Polish translation

Updated POTFILES.in and POTFILES.skip

Updated Czech translation

csd: Fix an oversight for popups

After the recent rework of client-side shadow code, menus,
tooltips and similar popups were ending up with solid decorations.
Fix this oversight.

Updated Hungarian translation

Update properties Hebrew translation

Updated Hebrew translation

Updated Greek translations

Updated Italian translation

Updated Italian translation

Stop using GDK_POINTER_MOTION_HINT_MASK in widgets

It is deprecated and no longer needed, and,as observed in
https://bugzilla.gnome.org/show_bug.cgi?id=746253 it
interferes with turning off event compression.

Silence another compiler warning

Again, our habit of comparing enum values against -1.

Don't return G_TYPE_INVALID as a pointer

Use NULL for g_return_val_if_fail checks if the function returns
a pointer. Pointed out by clang.

text view: Avoid clang compiler warnings

Again, our habit to use -1 as an unset value for enum types.

Remove a redundant const

const gchar const is still just const gchar, unless there's
indirections in between.

css: Silence a clang warning

Clang complains that this check can never be true. Since this
is a argument range check which we do to catch bad input,
convince clang to not complain instead of taking it out.

Remove a pointless comparison

Comparing an unsigned value against < 0 is not going to succeed.
Pointed out by clang.

paper size: Don't use abs() on floats

Use fabs() instead. Pointed out by clang.

More of the same

A similar case in GtkIconHelper. Here we were confusing ourselves,
and compared a icon_size against -1, although the variable is
never set to -1.

Avoid some clang compiler warnings

Avoid some compiler warnings from clang in deprecated code.
Most of these are around our habit of using -1 as an 'unset'
value for enumerations.

Fix some clang warnings

It complains about double const, and it is right.

Make the blur code compile with clang

clang complains that the expression involving sqrt() is not
constant, and thus refuses to accept it as a case label. So,
use precomputed values instead.

https://bugzilla.gnome.org/show_bug.cgi?id=746468

Fix the build

tooltip: Minor cleanup

Preserve the symmetry in gtk_tooltip_set_last_window.